Add AGENTS.md with Cursor Cloud development instructions#37
Conversation
Co-authored-by: Mathieu JESER <bobdivx@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: Mathieu JESER <bobdivx@users.noreply.github.com>
There was a problem hiding this comment.
Code Review
This pull request introduces AGENTS.md, a documentation file detailing the project's tech stack, commands, and development workflows for the Forge dashboard. The review feedback identifies a maintainability risk due to the lack of linting and testing frameworks and points out a discrepancy between the documented behavior of the Docker API endpoint and its actual implementation.
|
|
||
| ### Development notes | ||
|
|
||
| - **No ESLint/Prettier/test framework** is configured in this repo. The build (`npm run build`) is the primary correctness check. |
There was a problem hiding this comment.
The absence of ESLint, Prettier, and a testing framework is a significant maintainability risk.\n\nWhile documenting this state is helpful for agents, it highlights a major gap in the project's quality assurance. Relying solely on the build process for correctness checks can lead to regressions and inconsistent code style. It is highly recommended to integrate these tools to automate quality checks and ensure long-term project health.
| - The dev server auto-creates the SQLite database and seeds it on first run. No manual DB setup is needed. | ||
| - The app redirects unauthenticated users to `/login`. Register via the "Créer un compte" tab (min 10-char password). After registration, the setup wizard at `/setup` can be skipped. | ||
| - **OpenClaw** and **Ollama** are optional external services. The dashboard gracefully degrades when they are unreachable. | ||
| - The Docker-related API endpoint (`/api/docker.ts`) gracefully returns empty data when Docker is unavailable. |
There was a problem hiding this comment.
There is a discrepancy between this documentation and the implementation in src/pages/api/docker.ts.\n\nThe documentation states the endpoint "gracefully returns empty data when Docker is unavailable," but the code returns a 500 Internal Server Error if the docker command fails (e.g., if Docker is not installed or the daemon is not running). To achieve the documented "graceful degradation," the API should return an empty array [] with a 200 OK status on failure. If the current error behavior is intended, please update this documentation to reflect it.
| - The Docker-related API endpoint (`/api/docker.ts`) gracefully returns empty data when Docker is unavailable. | |
| - The Docker-related API endpoint (/api/docker.ts) returns an error response when Docker is unavailable (except on Vercel where it returns empty data). |
Summary
Adds
AGENTS.mdwith Cursor Cloud specific instructions for future agents working on this codebase.What's included
npm install,npm run dev,npm run build)astro checkmay hang in constrained environments/login→ setup wizard at/setup→ dashboardEnvironment setup verified
npm installnpm run devon port 4321npm run buildcompletes successfullyDemo
devforge_registration_and_dashboard_demo.mp4
Registration flow and dashboard navigation demo.
Dashboard overview
Agents management page
Projects page
To show artifacts inline, enable in settings.